Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(faq): add rule for issue #1861

Merged
merged 2 commits into from
Dec 21, 2017
Merged

docs(faq): add rule for issue #1861

merged 2 commits into from
Dec 21, 2017

Conversation

atian25
Copy link
Member

@atian25 atian25 commented Dec 21, 2017

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

1. 我们推荐如果是小问题(错别字修改,小的 bug fix)直接提交 PR。
2. 如果是一个新需求,请提供:详细需求描述,最好是有伪代码示意。
3. 如果是一个 BUG,请提供:复现步骤,错误日志以及相关配置,并尽量填写下面的模板中的条目。
4. **如果可以,仅可能使用 `egg-init --type=simple bug` 提供一个最小可复现的代码仓库,方便我们排查问题。**
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

尽可能

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@atian25 atian25 merged commit 4c1617a into master Dec 21, 2017
@atian25 atian25 deleted the docs-faq branch December 21, 2017 01:50
popomore pushed a commit that referenced this pull request Dec 21, 2017
docs(faq): add rule for issue (#1861)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants